baubs git - /page/commit.mu

Micron Document

baubs git

Node / mirrors / respira / commits / 77ec00d

Commit 77ec00df86b765da46a4bbbd6741fb1c68526e04

󰉖 Browse tree at this commit

Parents : 512eb73

Author : Jan-Henrik Bruhn <jan-henrik.bruhn@offis.de>

Date : 2025-12-27T17:11:31+01:00

fix: Remove ineffective React.memo from PatternCanvas

Address Copilot review feedback: PatternCanvas doesn't accept any props,

so React.memo has no effect. The component re-renders are driven by

Zustand store subscriptions which trigger regardless of memoization.

Keep React.memo on PatternLayer since it does receive props and benefits

from memoization.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Changes

1 files changed, 3 insertions(+), 3 deletions(-)

M src/components/PatternCanvas/PatternCanvas.tsx +3 -3

Diff

diff --git a/src/components/PatternCanvas/PatternCanvas.tsx b/src/components/PatternCanvas/PatternCanvas.tsx

index dbddabf..fae8a58 100644

--- a/src/components/PatternCanvas/PatternCanvas.tsx

+++ b/src/components/PatternCanvas/PatternCanvas.tsx

@@ -1,4 +1,4 @@

-import { useRef, useMemo, memo } from "react";

+import { useRef, useMemo } from "react";

import { useShallow } from "zustand/react/shallow";

import {

useMachineStore,

@@ -23,7 +23,7 @@ import { ZoomControls } from "./ZoomControls";

import { PatternLayer } from "./PatternLayer";

import { useCanvasViewport, usePatternTransform } from "@/hooks";

-export const PatternCanvas = memo(function PatternCanvas() {

+export function PatternCanvas() {

// Machine store

const { sewingProgress, machineInfo } = useMachineStore(

useShallow((state) => ({

@@ -277,4 +277,4 @@ export const PatternCanvas = memo(function PatternCanvas() {

</CardContent>

</Card>

);

-});